Skip to content

Conversation

@gregfromstl
Copy link
Contributor

@gregfromstl gregfromstl commented Nov 7, 2024

CNCT-2182
CNCT-2180
CNCT-2177
CNCT-2179
CNCT-2178
CNCT-2176

analytics.mp4

PR-Codex overview

This PR introduces new analytics features and components to enhance user statistics visualization within the dashboard. It adds new types for user statistics, updates existing components for better data handling, and refines the UI for displaying analytics.

Detailed summary

  • Added WalletUserStats interface for user statistics.
  • Updated ProjectOverviewHeader to include interval and range props.
  • Enhanced Stat component to display trends with badges.
  • Introduced PieChart, BarChart, and StatBreakdownCard components for data visualization.
  • Modified ProjectOverviewPage to fetch and display wallet and user analytics.
  • Refactored getWalletUsers, getUserOpUsage, and related API functions for improved data retrieval.
  • Updated existing stories and components for better integration with new features.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@linear
Copy link

linear bot commented Nov 7, 2024

@vercel
Copy link

vercel bot commented Nov 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 7, 2024 6:44pm
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-v2 ⬜️ Skipped (Inspect) Nov 7, 2024 6:44pm
thirdweb_playground ⬜️ Skipped (Inspect) Nov 7, 2024 6:44pm
wallet-ui ⬜️ Skipped (Inspect) Nov 7, 2024 6:44pm

@changeset-bot
Copy link

changeset-bot bot commented Nov 7, 2024

⚠️ No Changeset found

Latest commit: 9f9410a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 7, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@codecov
Copy link

codecov bot commented Nov 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.73%. Comparing base (2579525) to head (9f9410a).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5340   +/-   ##
=======================================
  Coverage   45.73%   45.73%           
=======================================
  Files        1070     1070           
  Lines       55634    55634           
  Branches     4074     4074           
=======================================
  Hits        25445    25445           
  Misses      29500    29500           
  Partials      689      689           
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from 2579525
packages 40.88% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 43 KB (0%) 861 ms (0%) 324 ms (-22.35% 🔽) 1.2 s
thirdweb (cjs) 103.05 KB (0%) 2.1 s (0%) 913 ms (-2.17% 🔽) 3 s
thirdweb (minimal + tree-shaking) 4.85 KB (0%) 97 ms (0%) 41 ms (+157.96% 🔺) 138 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 6 ms (-47.09% 🔽) 16 ms
thirdweb/react (minimal + tree-shaking) 17.56 KB (0%) 352 ms (0%) 187 ms (+106.67% 🔺) 538 ms

@vercel vercel bot temporarily deployed to Preview – thirdweb_playground November 7, 2024 16:50 Inactive
@vercel vercel bot temporarily deployed to Preview – docs-v2 November 7, 2024 16:50 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-ui November 7, 2024 16:50 Inactive
@gregfromstl gregfromstl requested a review from jnsdls November 7, 2024 16:50
@vercel vercel bot temporarily deployed to Preview – wallet-ui November 7, 2024 17:04 Inactive
@vercel vercel bot temporarily deployed to Preview – docs-v2 November 7, 2024 17:04 Inactive
@vercel vercel bot temporarily deployed to Preview – thirdweb_playground November 7, 2024 17:04 Inactive
@vercel vercel bot temporarily deployed to Preview – thirdweb_playground November 7, 2024 17:06 Inactive
@vercel vercel bot temporarily deployed to Preview – docs-v2 November 7, 2024 17:06 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-ui November 7, 2024 17:06 Inactive
@gregfromstl gregfromstl force-pushed the greg/cnct-2182-analytics-overview-page-build branch from 1eb43ab to 5e6c914 Compare November 7, 2024 17:07
@vercel vercel bot temporarily deployed to Preview – docs-v2 November 7, 2024 17:07 Inactive
@vercel vercel bot temporarily deployed to Preview – thirdweb_playground November 7, 2024 17:07 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-ui November 7, 2024 17:07 Inactive
@gregfromstl gregfromstl self-assigned this Nov 7, 2024
@gregfromstl gregfromstl added the Feature Adds new functionality to the public API. label Nov 7, 2024
@gregfromstl gregfromstl added the merge-queue Adds the pull request to Graphite's merge queue. label Nov 7, 2024
Copy link
Contributor Author

gregfromstl commented Nov 7, 2024

Merge activity

  • Nov 7, 1:21 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 7, 1:21 PM EST: A user added this pull request to the Graphite merge queue.
  • Nov 7, 1:36 PM EST: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'Unit Tests').
  • Nov 7, 1:37 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 7, 6:37 PM UTC: The merge label 'merge-queue' was removed. This PR will no longer be merged by the Graphite merge queue
  • Nov 7, 1:38 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 7, 6:39 PM UTC: The merge label 'merge-queue' was removed. This PR will no longer be merged by the Graphite merge queue
  • Nov 7, 1:39 PM EST: @graphite-app we removed the merge queue label because we could not find a Graphite account associated with your GitHub profile.

You must have a Graphite account in order to use the merge queue. Create an account and try again using this link

gregfromstl added a commit that referenced this pull request Nov 7, 2024
CNCT-2182
CNCT-2180
CNCT-2177
CNCT-2179
CNCT-2178
CNCT-2176

https://github.com/user-attachments/assets/5c676ec4-ca2f-4796-880a-944636f9f0d5

<!-- start pr-codex -->

---

## PR-Codex overview
This PR focuses on enhancing the analytics and user statistics features in the dashboard. It introduces new components, updates existing ones, and improves the handling of wallet and user data, including stats visualization and data fetching.

### Detailed summary
- Added `WalletUserStats` interface to manage user statistics.
- Updated `ProjectOverviewHeader` to accept `interval` and `range` props.
- Introduced new `Stat` component for displaying metrics with trends.
- Created `PieChart`, `BarChart`, and `CombinedBarChartCard` components for visualizing data.
- Implemented `StatBreakdownCard` for detailed metric breakdowns.
- Enhanced `UsersChartCard` to process and display user statistics.
- Updated `PageProps` and data fetching functions for improved analytics handling.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->
@gregfromstl gregfromstl force-pushed the greg/cnct-2182-analytics-overview-page-build branch from 5e6c914 to 4d975f6 Compare November 7, 2024 18:22
@vercel vercel bot temporarily deployed to Preview – docs-v2 November 7, 2024 18:22 Inactive
@vercel vercel bot temporarily deployed to Preview – thirdweb_playground November 7, 2024 18:22 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-ui November 7, 2024 18:22 Inactive
CNCT-2182
CNCT-2180
CNCT-2177
CNCT-2179
CNCT-2178
CNCT-2176

https://github.com/user-attachments/assets/5c676ec4-ca2f-4796-880a-944636f9f0d5

<!-- start pr-codex -->

---

## PR-Codex overview
This PR focuses on enhancing the analytics components in the dashboard by introducing new stats interfaces, updating existing components to utilize these stats, and improving the layout and functionality of various charts and cards.

### Detailed summary
- Added `WalletUserStats` interface.
- Updated `ProjectOverviewHeader` to accept `interval` and `range`.
- Enhanced `Stat` component to display trends with badges.
- Introduced `PieChart`, `BarChart`, and `CombinedBarChartCard` components.
- Updated `EmptyState` messaging.
- Improved layout in several components for better responsiveness.
- Added new stories for `PieChartCard`, `StatBreakdownCard`, and others in Storybook.
- Refactored data fetching functions to support new stats.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->
@gregfromstl gregfromstl force-pushed the greg/cnct-2182-analytics-overview-page-build branch from 4d975f6 to 9f9410a Compare November 7, 2024 18:33
@vercel vercel bot temporarily deployed to Preview – docs-v2 November 7, 2024 18:33 Inactive
@vercel vercel bot temporarily deployed to Preview – thirdweb_playground November 7, 2024 18:33 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-ui November 7, 2024 18:33 Inactive
@graphite-app graphite-app bot added merge-queue Adds the pull request to Graphite's merge queue. and removed merge-queue Adds the pull request to Graphite's merge queue. labels Nov 7, 2024
@graphite-app graphite-app bot merged commit 9f9410a into main Nov 7, 2024
32 checks passed
@graphite-app graphite-app bot deleted the greg/cnct-2182-analytics-overview-page-build branch November 7, 2024 19:10
@vercel vercel bot temporarily deployed to Production – wallet-ui November 7, 2024 19:11 Inactive
@vercel vercel bot temporarily deployed to Production – thirdweb_playground November 7, 2024 19:11 Inactive
@vercel vercel bot temporarily deployed to Production – docs-v2 November 7, 2024 19:11 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dashboard Involves changes to the Dashboard. Feature Adds new functionality to the public API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants